Skip to content

feat(bulk-import): add documentation for Bulk Import Scaffolder template parameters #1667

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Nov 4, 2025
Merged

feat(bulk-import): add documentation for Bulk Import Scaffolder template parameters #1667

merged 1 commit into from
Nov 4, 2025

Conversation

@AndrienkoAleksandr
Copy link
Contributor

@AndrienkoAleksandr AndrienkoAleksandr commented Oct 16, 2025

Hey, I just made a Pull Request!

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

@rhdh-gh-app
Copy link

rhdh-gh-app bot commented Oct 16, 2025

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @red-hat-developer-hub/backstage-plugin-bulk-import-backend

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-bulk-import-backend workspaces/bulk-import/plugins/bulk-import-backend none v6.5.1

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 16, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

AndrienkoAleksandr
AndrienkoAleksandr commented Oct 17, 2025
View reviewed changes
Copy link
Contributor Author

@AndrienkoAleksandr AndrienkoAleksandr left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hope it will be merged soon.

@imykhno
Copy link
Contributor

imykhno commented Oct 27, 2025

It's look fine to me

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm label Oct 27, 2025
dzemanov
dzemanov requested changes Oct 27, 2025
View reviewed changes
Copy link
Member

@dzemanov dzemanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AndrienkoAleksandr do we need a changeset since the change is in README?

@AndrienkoAleksandr
Copy link
Contributor Author

AndrienkoAleksandr commented Nov 4, 2025

@AndrienkoAleksandr do we need a changeset since the change is in README?

@dzemanov, I didn’t change any code or dependencies — this PR only updates documentation.
I don’t think we need a changeset and a new release for this.

dzemanov
dzemanov approved these changes Nov 4, 2025
View reviewed changes
Copy link
Member

@dzemanov dzemanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AndrienkoAleksandr I am asking because of this comment and the fact that README is part of the plugin. But it will be updated when we do other changes, so this is good with me.

@AndrienkoAleksandr AndrienkoAleksandr merged commit 7c2a1c6 into redhat-developer:main Nov 4, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dzemanov dzemanov dzemanov approved these changes

@rm3l rm3l Awaiting requested review from rm3l rm3l is a code owner

@debsmita1 debsmita1 Awaiting requested review from debsmita1 debsmita1 is a code owner

@its-mitesh-kumar its-mitesh-kumar Awaiting requested review from its-mitesh-kumar its-mitesh-kumar is a code owner

Assignees

@imykhno imykhno

Labels

lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@AndrienkoAleksandr @imykhno @dzemanov